Skip to content

add a new workflow task failure cause for workflow pause#821

Merged
spkane31 merged 3 commits into
mainfrom
spk/fail-not-started-wft
Jul 10, 2026
Merged

add a new workflow task failure cause for workflow pause#821
spkane31 merged 3 commits into
mainfrom
spk/fail-not-started-wft

Conversation

@spkane31

@spkane31 spkane31 commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

What changed?
Add a new WorkflowTaskFailedCause enum WORKFLOW_TASK_FAILED_CAUSE_NOT_STARTED_TASK_PAUSED_WORKFLOW

Why?
A new failure cause when a workflow task is scheduled but never started when a workflow is paused by an operator. The workflow task will be failed with this new cause.

Breaking changes
N/A

Server PR
Not needed

@spkane31 spkane31 requested review from a team July 8, 2026 23:06
@spkane31 spkane31 force-pushed the spk/fail-not-started-wft branch from d9e4cab to 6c16e49 Compare July 8, 2026 23:33
@spkane31 spkane31 requested review from simvlad and yycptt July 9, 2026 17:59
WORKFLOW_TASK_FAILED_CAUSE_EXTERNAL_STORAGE_FAILURE = 38;
// A workflow task is failed because the workflow is paused before the task is started.
WORKFLOW_TASK_FAILED_CAUSE_NOT_STARTED_TASK_PAUSED_WORKFLOW = 39;
WORKFLOW_TASK_FAILED_CAUSE_WORKFLOW_PAUSE_REQUESTED_BEFORE_TASK_STARTED = 40;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it intentional that there's 2 added here? PR description only mentions WORKFLOW_TASK_FAILED_CAUSE_NOT_STARTED_TASK_PAUSED_WORKFLOW. And they kinda seem like the same error but worded differently?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed. I messed up a previous rebase.

@spkane31 spkane31 force-pushed the spk/fail-not-started-wft branch from 3f8fcb9 to feba120 Compare July 10, 2026 16:31

@yuandrew yuandrew left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR description should be updated with the new name

@spkane31 spkane31 merged commit ee3ba7a into main Jul 10, 2026
4 checks passed
@spkane31 spkane31 deleted the spk/fail-not-started-wft branch July 10, 2026 16:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants